Skip to content

Conversation

@jsonbailey
Copy link
Contributor

@jsonbailey jsonbailey commented Jan 22, 2026

Note

Strengthens coverage for data store behavior, persistence interactions, and FDv2 integration.

  • InMemoryFeatureStoreV2 tests: initialization, string/symbol key compatibility for get, all filtering (excludes deleted), set_basis replacement/clearing/error handling, apply_delta (adds/updates/deletes, empty deltas), and thread-safety under concurrent reads/writes.
  • Store tests: applies TRANSFER_FULL via set_basis, TRANSFER_CHANGES via apply_delta, handles TRANSFER_NONE, processes DELETE, broadcasts change sets, and commit behavior with/without persistent store including encoding, failures, and read-only mode.
  • FDv2 persistence tests: verify delta upserts in READ_WRITE, no writes in READ_ONLY, persistence across initializer vs synchronizer, recovery/flush on stale store, and presence/behavior of data store status provider.

Written by Cursor Bugbot for commit e47fdc1. This will update automatically on new commits. Configure here.

@jsonbailey jsonbailey marked this pull request as ready for review January 22, 2026 20:58
@jsonbailey jsonbailey requested a review from a team as a code owner January 22, 2026 20:58
Base automatically changed from jb/sdk-1547/test-persistent-data-store-recovery to main January 23, 2026 18:00
@jsonbailey jsonbailey merged commit 74a0f88 into main Jan 23, 2026
10 checks passed
@jsonbailey jsonbailey deleted the jb/sdk-1548/atomic-memory-store-tests branch January 23, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants